-
Notifications
You must be signed in to change notification settings - Fork 28
fix: coffee agentcy docs with new SLIM group communication changes #271
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
d4b90b9 to
0d9516a
Compare
Signed-off-by: Shridhar Shah <[email protected]>
Signed-off-by: Shridhar Shah <[email protected]>
Signed-off-by: Shridhar Shah <[email protected]>
15e1a3e to
91850cd
Compare
muscariello
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure this is the right way. SLIM v0.6 but also v0.5 supports tight integration w/ RPC protocols. I am not sure if pubsub as currently used makes a lot of sense.
|
Yep good call out, we saw that in 0.5 and 0.6 — we haven’t moved to it yet. Once we get there, we’ll definitely look at pub/sub again. Would it be a problem in your view to keep it for 0.4? I'm feeling a new issue to go over what we all want to see in 0.6 integration. |
Signed-off-by: Aron Kerekes <[email protected]>
|
Is this effected by the latest slim release? v0.6.0? |
Update CoffeeAgntcy docs with new changes in SLIM v0.4.0 and group communication